Skip to content

Make Android app shortcut use 'Home Assistant' as name.#8261

Merged
fabaff merged 1 commit into
home-assistant:devfrom
michaelarnauts:app_name
Jun 30, 2017
Merged

Make Android app shortcut use 'Home Assistant' as name.#8261
fabaff merged 1 commit into
home-assistant:devfrom
michaelarnauts:app_name

Conversation

@michaelarnauts
Copy link
Copy Markdown
Contributor

Currently, when you create a shortcut to Home Assistant on Android, it suggests the name "Assistant" instead of "Home Assistant".

I think "Home Assistant" is the correct name for the app to clearly differentiate with Google Assistant or other Assistants.

Check out the animation on this page to see the "Assistant" name.

Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🐦

@fabaff fabaff merged commit 0bcb783 into home-assistant:dev Jun 30, 2017
@balloob balloob mentioned this pull request Jul 1, 2017
balloob added a commit that referenced this pull request Jul 1, 2017
balloob added a commit that referenced this pull request Jul 1, 2017
* Revert "Version bump to 0.49.0.dev0 (#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (#8261)"

This reverts commit 0bcb783.
@balloob
Copy link
Copy Markdown
Member

balloob commented Jul 1, 2017

This should not have been merged and needs more discussion. The reason the name is "Assistant" is because otherwise the name will be cutoff on the home screen and it will be "Home Ass…"

@balloob
Copy link
Copy Markdown
Member

balloob commented Jul 1, 2017

(that's also why the property is called short_name)

balloob added a commit that referenced this pull request Jul 1, 2017
* Revert "Version bump to 0.49.0.dev0 (#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (#8261)"

This reverts commit 0bcb783.
@michaelarnauts
Copy link
Copy Markdown
Contributor Author

Hmm, it's "Home Assi.." on default font size here, and "Home Assista.." on small font.

The downside is that Android always uses the short name, like in the task switcher where there is enough space for the full name.

I've gotten used to all the abreviations on the home screen that I haven't noticed it's abreviated. I did notice the "Assistant" name wasn't actually correct.

dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
…e-assistant#8271)

* Revert "Version bump to 0.49.0.dev0 (home-assistant#8266)"

This reverts commit 8e4394f.

* Revert "Adding done_message to alert (home-assistant#8116)"

This reverts commit 5e56bc7.

* Revert "Camera services arm disarm including Netgear Arlo (home-assistant#7961)"

This reverts commit ed20f7e.

* Revert "Make Android app shortcut use 'Home Assistant' as name instead of just 'Assistant'. (home-assistant#8261)"

This reverts commit 0bcb783.
@home-assistant home-assistant locked and limited conversation to collaborators Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants